added c-interface (lapacke) support#699
added c-interface (lapacke) support#699DirkdeDraak wants to merge 1 commit intoeasybuilders:developfrom DirkdeDraak:lapack-with-c-interface
Conversation
|
Automatic reply from Jenkins: Can I test this? |
|
Jenkins: ok to test |
|
looks fine, but we'll need to make sure this works with all existing LAPACK easyconfig we have (there should be no regressions) I'll take a look at that. |
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1218/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
| self.cfg.update('buildopts', 'lib') | ||
|
|
||
|
|
||
| if os.path.exists(os.path.join(self.cfg['start_dir'], 'lapacke')): |
There was a problem hiding this comment.
how about just checking the version? this should be there in LAPACK 3.4.0 and up
There was a problem hiding this comment.
see https://github.com/DirkdeDraak/easybuild-easyblocks/pull/1, if you merge my PR to your branch, it'll be included here
|
@DirkdeDraak: hmm, this doesn't work as expected... It results in running this: which yields: because you can't specify two targets ( |
4 participants
No description provided.